[PATCH] D85345: [BuildLibCalls] Add noundef to standard I/O functions

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 5 20:32:47 PDT 2020


jdoerfert added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/BuildLibCalls.cpp:509
+    Changed |= setDoesNotThrow(F);
+    Changed |= setDoesNotCapture(F, 1);
+    return Changed;
----------------
I'd split it into noundef and other attributes. noundef is so new that we should keep that separate, having the others in a single patch is fine with me though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85345



More information about the llvm-commits mailing list