[PATCH] D83744: [Attributor] Attributor call site specific AAValueConstantRange

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 11 06:23:54 PST 2021


RKSimon added inline comments.


================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:1138
+    }
+#endif
+    if (!shouldPropagateCallBaseContext(IRP))
----------------
@kuter This is breaking some EXPENSIVE_CHECKS builds such as http://lab.llvm.org:8011/#/builders/16/builds/7624

AbstractAttribute isn't defined until later in the header - oddly my MSVC build is quite happy (after my fix at rGe74d6269259e28225a494ce98bd1762cdac1fe89)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83744/new/

https://reviews.llvm.org/D83744



More information about the llvm-commits mailing list