pghs975uc
/
infer_clone
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity
You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
Branches Tags
${ item.name }
Create tag ${ searchTerm }
Create branch ${ searchTerm }
from 'fbe1c92c17'
${ noResults }
infer_clone/infer/tests/codetoanalyze/cpp/shared/reference/unbox.cpp.dot

91 lines
6.4 KiB
Raw Normal View History Unescape

Add @generated comment to icfg.dot files Reviewed By: jvillard Differential Revision: D3358243 fbshipit-source-id: a47cc01
9 years ago
/* @generated */
[cfg] rename iCFG to cfg in dotty files Summary: Not sure what an "iCFG" is but the dotty is only about CFGs anyway. Diff obtained by mass-`sed`. Reviewed By: sblackshear Differential Revision: D6324280 fbshipit-source-id: b7603bb
7 years ago
digraph cfg {
Get rid of Declare_locals Reviewed By: jeremydubreil Differential Revision: D9234331 fbshipit-source-id: 70443eabe
6 years ago
"fun_p#7468829763884786220.ff1d58f26970dcc8ee3c2a153c5a5b85_1" [label="1: Start fun_p\nFormals: p:int*\nLocals: \n " color=yellow style=filled]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"fun_p#7468829763884786220.ff1d58f26970dcc8ee3c2a153c5a5b85_1" -> "fun_p#7468829763884786220.ff1d58f26970dcc8ee3c2a153c5a5b85_3" ;
"fun_p#7468829763884786220.ff1d58f26970dcc8ee3c2a153c5a5b85_2" [label="2: Exit fun_p \n " color=yellow style=filled]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"fun_p#7468829763884786220.ff1d58f26970dcc8ee3c2a153c5a5b85_3" [label="3: Return Stmt \n n$0=*&p:int* [line 8, column 29]\n n$1=*n$0:int [line 8, column 28]\n *&return:int=n$1 [line 8, column 21]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"fun_p#7468829763884786220.ff1d58f26970dcc8ee3c2a153c5a5b85_3" -> "fun_p#7468829763884786220.ff1d58f26970dcc8ee3c2a153c5a5b85_2" ;
Get rid of Declare_locals Reviewed By: jeremydubreil Differential Revision: D9234331 fbshipit-source-id: 70443eabe
6 years ago
"fun_r#8688550998084520100.a539308a01e8443f65be5d44c29a73f6_1" [label="1: Start fun_r\nFormals: p:int&\nLocals: \n " color=yellow style=filled]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"fun_r#8688550998084520100.a539308a01e8443f65be5d44c29a73f6_1" -> "fun_r#8688550998084520100.a539308a01e8443f65be5d44c29a73f6_3" ;
"fun_r#8688550998084520100.a539308a01e8443f65be5d44c29a73f6_2" [label="2: Exit fun_r \n " color=yellow style=filled]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"fun_r#8688550998084520100.a539308a01e8443f65be5d44c29a73f6_3" [label="3: Return Stmt \n n$0=*&p:int& [line 10, column 28]\n n$1=*n$0:int [line 10, column 28]\n *&return:int=n$1 [line 10, column 21]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"fun_r#8688550998084520100.a539308a01e8443f65be5d44c29a73f6_3" -> "fun_r#8688550998084520100.a539308a01e8443f65be5d44c29a73f6_2" ;
Get rid of Declare_locals Reviewed By: jeremydubreil Differential Revision: D9234331 fbshipit-source-id: 70443eabe
6 years ago
"fun_v#125358748374922080.2b082c989a86eb6a918b15eb596c685a_1" [label="1: Start fun_v\nFormals: p:int\nLocals: \n " color=yellow style=filled]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"fun_v#125358748374922080.2b082c989a86eb6a918b15eb596c685a_1" -> "fun_v#125358748374922080.2b082c989a86eb6a918b15eb596c685a_3" ;
"fun_v#125358748374922080.2b082c989a86eb6a918b15eb596c685a_2" [label="2: Exit fun_v \n " color=yellow style=filled]
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"fun_v#125358748374922080.2b082c989a86eb6a918b15eb596c685a_3" [label="3: Return Stmt \n n$0=*&p:int [line 9, column 27]\n *&return:int=n$0 [line 9, column 20]\n " shape="box"]
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"fun_v#125358748374922080.2b082c989a86eb6a918b15eb596c685a_3" -> "fun_v#125358748374922080.2b082c989a86eb6a918b15eb596c685a_2" ;
Get rid of Declare_locals Reviewed By: jeremydubreil Differential Revision: D9234331 fbshipit-source-id: 70443eabe
6 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_1" [label="1: Start unbox_ptr\nFormals: \nLocals: p:int* a:int \n " color=yellow style=filled]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[clang] update clang plugin to hash mangled names Summary: With this change and the previous facebook-clang-plugins change, infer no longer exhausts the biniou buffer when reading the serialized C++ AST. update-submodule: facebook-clang-plugins Reviewed By: mbouaziz Differential Revision: D5891081 fbshipit-source-id: cf48eac
7 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_1" -> "unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_7" ;
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_2" [label="2: Exit unbox_ptr \n " color=yellow style=filled]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_3" [label="3: Call _fun_fun_r \n n$0=*&p:int* [line 31, column 10]\n n$1=_fun_fun_r(n$0:int&) [line 31, column 3]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[clang] update clang plugin to hash mangled names Summary: With this change and the previous facebook-clang-plugins change, infer no longer exhausts the biniou buffer when reading the serialized C++ AST. update-submodule: facebook-clang-plugins Reviewed By: mbouaziz Differential Revision: D5891081 fbshipit-source-id: cf48eac
7 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_3" -> "unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_2" ;
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_4" [label="4: Call _fun_fun_v \n n$2=*&p:int* [line 30, column 10]\n n$3=*n$2:int [line 30, column 9]\n n$4=_fun_fun_v(n$3:int) [line 30, column 3]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[clang] update clang plugin to hash mangled names Summary: With this change and the previous facebook-clang-plugins change, infer no longer exhausts the biniou buffer when reading the serialized C++ AST. update-submodule: facebook-clang-plugins Reviewed By: mbouaziz Differential Revision: D5891081 fbshipit-source-id: cf48eac
7 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_4" -> "unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_3" ;
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_5" [label="5: Call _fun_fun_p \n n$5=*&p:int* [line 29, column 9]\n n$6=_fun_fun_p(n$5:int*) [line 29, column 3]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[clang] update clang plugin to hash mangled names Summary: With this change and the previous facebook-clang-plugins change, infer no longer exhausts the biniou buffer when reading the serialized C++ AST. update-submodule: facebook-clang-plugins Reviewed By: mbouaziz Differential Revision: D5891081 fbshipit-source-id: cf48eac
7 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_5" -> "unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_4" ;
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_6" [label="6: DeclStmt \n VARIABLE_DECLARED(p:int*); [line 27, column 3]\n *&p:int*=&a [line 27, column 3]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[clang] update clang plugin to hash mangled names Summary: With this change and the previous facebook-clang-plugins change, infer no longer exhausts the biniou buffer when reading the serialized C++ AST. update-submodule: facebook-clang-plugins Reviewed By: mbouaziz Differential Revision: D5891081 fbshipit-source-id: cf48eac
7 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_6" -> "unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_5" ;
[SIL] change `variable_initialization()` builtin to a new auxiliary instruction Summary: Instead of emitting an ad-hoc builtin on variable declaration emit a new metadata instruction. This allows us to remove the code matching on that ad-hoc builtin that had to be inserted in several checkers. Inferbo & pulse used that information meaningfully and had to undergo some minor changes to cope with the new metada instruction. Reviewed By: ezgicicek Differential Revision: D14833100 fbshipit-source-id: 9b3009d22
6 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_7" [label="7: DeclStmt \n VARIABLE_DECLARED(a:int); [line 26, column 3]\n *&a:int=3 [line 26, column 3]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[clang] update clang plugin to hash mangled names Summary: With this change and the previous facebook-clang-plugins change, infer no longer exhausts the biniou buffer when reading the serialized C++ AST. update-submodule: facebook-clang-plugins Reviewed By: mbouaziz Differential Revision: D5891081 fbshipit-source-id: cf48eac
7 years ago
"unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_7" -> "unbox_ptr#3550280956167916174.75d50cc2e2dfffd1cc23613b01fc878b_6" ;
Get rid of Declare_locals Reviewed By: jeremydubreil Differential Revision: D9234331 fbshipit-source-id: 70443eabe
6 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_1" [label="1: Start unbox_ref\nFormals: \nLocals: r:int& a:int \n " color=yellow style=filled]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_1" -> "unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_7" ;
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_2" [label="2: Exit unbox_ref \n " color=yellow style=filled]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_3" [label="3: Call _fun_fun_r \n n$0=*&r:int& [line 21, column 9]\n n$1=_fun_fun_r(n$0:int&) [line 21, column 3]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_3" -> "unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_2" ;
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_4" [label="4: Call _fun_fun_v \n n$2=*&r:int& [line 20, column 9]\n n$3=*n$2:int [line 20, column 9]\n n$4=_fun_fun_v(n$3:int) [line 20, column 3]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_4" -> "unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_3" ;
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_5" [label="5: Call _fun_fun_p \n n$5=*&r:int& [line 19, column 10]\n n$6=_fun_fun_p(n$5:int*) [line 19, column 3]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_5" -> "unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_4" ;
[cfg] run pre-analysis lazily in ondemand Summary: This also prints the CFGs *after* pre-analysis for individual procedures in infer-out/captured/<filename>/<proc>.dot. One can also look up the CFGs before pre-analysis in infer-out/captured/proc_cfgs_frontend.dot. Context: I want to add a pre-analysis that needs to look at proc attributes inter-procedurally. For this to make sense it has to happen *after* all of capture, and before analysis. Thus, this diff brings back the lazy running of the pre-analysis like in D15803492, except that we still make sure to run the pre-analyses systematically regardless of the checkers being run by running the pre-analysis from ondemand.ml. Also we don't need to re-introduce the "did_preanalysis" proc attribute for the same reason that the pre-analysis is now run once and for all by ondemand.ml (instead of each individual checker back in the days). This has the benefit of running the pre-analysis only when needed, and the drawback that several concurrent processes analysing the same proc descs will duplicate work. Since pre-analyses are supposed to be very fast I assume that neither is a big deal. If they become more expensive then the benefit gets bigger and the drawback is just the same as with regular analyses. Reviewed By: skcho Differential Revision: D18573920 fbshipit-source-id: de350eaef
5 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_6" [label="6: DeclStmt \n VARIABLE_DECLARED(r:int&); [line 17, column 3]\n *&r:int&=&a [line 17, column 3]\n " shape="box"]
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_6" -> "unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_5" ;
[SIL] change `variable_initialization()` builtin to a new auxiliary instruction Summary: Instead of emitting an ad-hoc builtin on variable declaration emit a new metadata instruction. This allows us to remove the code matching on that ad-hoc builtin that had to be inserted in several checkers. Inferbo & pulse used that information meaningfully and had to undergo some minor changes to cope with the new metada instruction. Reviewed By: ezgicicek Differential Revision: D14833100 fbshipit-source-id: 9b3009d22
6 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_7" [label="7: DeclStmt \n VARIABLE_DECLARED(a:int); [line 16, column 3]\n *&a:int=3 [line 16, column 3]\n " shape="box"]
Sort nodes when printing cfg to dot file Summary: Currently cfg nodes are written into dot files in whatever order they appear in a hash table. This seems unnecessarily sensitive, so this diff sorts the nodes. Reviewed By: dulmarod Differential Revision: D4232377 fbshipit-source-id: a907cc6
8 years ago
[debug] print procedures in alphabetical order in cfgs Summary: When looking at large CFGs, at least in `xdot`, it's often difficult to find the procedure you're looking for. Sorting the proc names puts them in alphabetical order, which makes searching one procedure easier. Reviewed By: mbouaziz Differential Revision: D7758521 fbshipit-source-id: 8e9997f
7 years ago
"unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_7" -> "unbox_ref#9977470601320200599.91094dce9e5b43dc4c89abcbc69b2c70_6" ;
Fix handling of C++ references Summary: @​public This diff changes following things: 1. expression_info.type_ptr has type than decl_ref_info.type_ptr for reference types. Use type from decl_ref_info as a source of truth 2. reference types need to have one extra dereference that is not in AST. Add handling for this. 3. [small refactor] create function that creates temporary variable from res_trans expression and returns new res_trans. Some caveats: 1. types are not quite right yet (see .dot files). 2. decl_ref_info might not be set for DeclRefExpr, make frontend crash in that case to catch when this happens This is high risk change since it changes behavior of every translation on very widely used expr. Reviewed By: @dulmarod Differential Revision: D2540632 fb-gh-sync-id: aa28936
9 years ago
}