Displaying 3 results from an estimated 3 matches for "hl_predicateloops".
2009 Aug 07
0
[LLVMdev] [PATCH] PR2218
...= CallSite::get(C);
+
+ // Pointer must be a parameter (case 1)
+ for (argI = 0; argI < CS.arg_size(); ++argI)
+ if (CS.getArgument(argI)->stripPointerCasts() == pointer)
+ break;
+
+ if (argI == CS.arg_size())
+ return false;
per http://llvm.org/docs/CodingStandards.html#hl_predicateloops
+ // Store cannot be volatile (case 2) and must-alias with our
pointer.
+ if (S->isVolatile()) {
+ return false;
+ } else {
no need for the 'else after return': http://llvm.org/docs/CodingStandards.html#hl_else_after_return
+ AliasAnalysis& AA = getAnalysis&l...
2009 Jul 25
2
[LLVMdev] [PATCH] PR2218
Hello,
Sorry for my stupid mistakes. I hope that everything is fine now. This
patch fixes PR2218. There are no loads in example, however
"instcombine" changes memcpy() into store/load.
Regards,
Jakub Staszak
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pr2218-2.patch
Type: application/octet-stream
Size: 6525 bytes
Desc: not available
URL:
2009 Sep 02
2
[LLVMdev] [PATCH] PR2218
...must be a parameter (case 1)
> + for (argI = 0; argI < CS.arg_size(); ++argI)
> + if (CS.getArgument(argI)->stripPointerCasts() == pointer)
> + break;
> +
> + if (argI == CS.arg_size())
> + return false;
>
> per http://llvm.org/docs/CodingStandards.html#hl_predicateloops
>
>
> + // Store cannot be volatile (case 2) and must-alias with our
> pointer.
> + if (S->isVolatile()) {
> + return false;
> + } else {
>
> no need for the 'else after return': http://llvm.org/docs/CodingStandards.html#hl_else_after_return
>...